The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Feb 7, 2026 8:18pm

@cursor review

Greptile Overview

Greptile Summary

Summary

This PR addresses an edge case where a socket entry can expire via the 1-hour TTL before the corresponding room presence is cleaned up, leaving stale presence in rooms. The changes appear to add cleanup paths in the socket connection/workflow handlers and adjust the room managers/types/feature-flag configuration so that presence is removed when the underlying socket mapping is missing or has been evicted.

Overall, the change fits into the existing socket/rooms architecture by extending the room memory-manager / redis-manager cleanup logic and wiring it through the connection/workflow handlers so room presence stays consistent with the sockets index, even when TTL-based eviction happens asynchronously.

Confidence Score: 3/5

• This PR is close to safe to merge, but has a correctness issue in workflow socket cleanup that can disconnect the wrong user under shared tabSessionId conditions.
• Score reduced due to a definite cross-user cleanup bug introduced in the workflow handler; the rest of the changes appear localized to presence cleanup and type/signature updates with no other confirmed issues found.
• apps/sim/socket/handlers/workflow.ts

Important Files Changed

Sequence Diagram

sequenceDiagram
  participant Client
  participant SocketHandlers as Socket Handlers
  participant Rooms as Rooms Manager
  participant Redis as Redis (sockets + presence)

  Client->>SocketHandlers: Connect / Resume
  SocketHandlers->>Redis: Read socket entry (TTL-backed)
  alt socket entry present
    SocketHandlers->>Rooms: Join / update presence
    Rooms->>Redis: Write presence
  else socket entry missing (expired by 1hr TTL)
    SocketHandlers->>Rooms: Cleanup presence for stale socket
    Rooms->>Redis: Remove presence + room membership
  end

  Client->>SocketHandlers: Workflow events
  SocketHandlers->>Redis: Resolve socket entry
  alt missing/expired
    SocketHandlers->>Rooms: Cleanup stale presence
    Rooms->>Redis: Remove presence
  else present
    SocketHandlers->>Rooms: Update presence
    Rooms->>Redis: Persist presence
  end

@cursor review